Skip to content

Comments

Change error handling for tool call#2

Merged
mark19891107 merged 2 commits intomainfrom
codex/change-error-info-to-content-property
Jun 20, 2025
Merged

Change error handling for tool call#2
mark19891107 merged 2 commits intomainfrom
codex/change-error-info-to-content-property

Conversation

@mark19891107
Copy link
Owner

Summary

  • return tool call errors inside the content field so MCP clients can read them
  • update tests accordingly

Testing

  • go test ./... (fails: Forbidden for proxy.golang.org)

https://chatgpt.com/codex/tasks/task_e_6855570abba4832ba20b6d8ef412884f

@mark19891107
Copy link
Owner Author

把 Call Tool 的 Error Message 也放到 Content 中,讓 MCP Client 可以讀取錯誤訊息

@mark19891107 mark19891107 merged commit d979584 into main Jun 20, 2025
1 check passed
@mark19891107 mark19891107 deleted the codex/change-error-info-to-content-property branch June 20, 2025 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant